Overhaul cookbook workflows around Jupyter Book 2 and BinderHub#167
Open
agoose77 wants to merge 1 commit into
Open
Overhaul cookbook workflows around Jupyter Book 2 and BinderHub#167agoose77 wants to merge 1 commit into
agoose77 wants to merge 1 commit into
Conversation
Contributor
|
Should've checked here, huh. |
agoose77
changed the title
This was referenced May 11, 2026
brian-rose
reviewed
May 11, 2026
Member
brian-rose
left a comment
brian-rose
left a comment
There was a problem hiding this comment.
I need to spend more time on a deeper dive here, but I really like what I see here so far. I have one specific comment about the hard-coding of the binderhub URL.
| # This action sets up evironment variables for e.g. mystmd to recognise and use | ||
| uses: 2i2c-org/clinder@action-v1 | ||
| with: | ||
| hub-url: https://binder.projectpythia.org/ |
Member
There was a problem hiding this comment.
This is the one place where I think it would be great to avoid hard-coding the solution, in anticipation of more bespoke project-specific binderhubs and/or use of these workflows outside the scope of Project Pythia.
There is a binderhub URL that lives in the myst config as
project:
jupyter:
binder:
url: https://binder.projectpythia.org/
which is actually defined by default in our upstream config file here.
Is there a simple way to parse the myst config to get this URL?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Following from the Statement of Work that 2i2c have authored to build out a replacement for BinderBot, this PR sets up the build action to use the https://github.com/2i2c-org/clinder action. It makes the following changes:
myst.yamltags, validated by a schema.mainbranch to GitHub Pages.build.shrecognition for custom builds.The new action simply sets up a BinderHub session and exposes it to the action runner via environment variables (by default). The MyST Document Engine is natively aware of these, and will use them to find the Jupyter Server instead of launching a local one.
I'm testing this over at https://github.com/ProjectPythia/ptype-ml-clinder-test
Note
I had some trouble migrating existing repositories — it seems that most cookbooks are broken in one way, or otherwise need environment variables.